Skip to content

west: mbedtls: Bump to 3.6.4 #94596

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Sep 6, 2025
Merged

west: mbedtls: Bump to 3.6.4 #94596

merged 1 commit into from
Sep 6, 2025

Conversation

@ceolin
Copy link
Member

@ceolin ceolin commented Aug 18, 2025 •
edited by fabiobaltieri
Loading

Bump mbetls to 3.6.4.

Fixes: #94924

@zephyrbot zephyrbot added the size: XS A PR changing only a single line of code label Aug 18, 2025
@zephyrbot
Copy link

zephyrbot commented Aug 18, 2025 •
edited
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
mbedtls zephyrproject-rtos/mbedtls@5f88993 zephyrproject-rtos/mbedtls@85440ef (zephyr) zephyrproject-rtos/[email protected]

Additional metadata changed:

Name URL Submodules West cmds module.yml
mbedtls ✏

⛔ DNM label due to: 1 project with metadata changes

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@zephyrbot zephyrbot added manifest manifest-mbedtls DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Aug 18, 2025
@ceolin ceolin linked an issue Aug 25, 2025 that may be closed by this pull request
mbedtls: update to 3.6.4 in 4.1 branch #94924
Open
ceolin added a commit to ceolin/zephyr that referenced this pull request Aug 25, 2025
@ceolin
west: mbedtls: Bump to 3.6.4
f9c3732 
Bump mbetls to 3.6.4.

Fixes: zephyrproject-rtos#94596

Signed-off-by: Flavio Ceolin <[email protected]>
kartben
kartben previously approved these changes Aug 25, 2025
View reviewed changes
tomi-font
tomi-font previously approved these changes Aug 26, 2025
View reviewed changes
Copy link
Contributor

@tomi-font tomi-font left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A release notes entry?

@fabiobaltieri
Copy link
Member

fabiobaltieri commented Aug 26, 2025

A release notes entry?

We kindof don't bother unless it's considered a major or at least notable feature at this point.

@fabiobaltieri fabiobaltieri removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Aug 26, 2025
valeriosetti
valeriosetti previously approved these changes Aug 28, 2025
View reviewed changes
ceolin added a commit to ceolin/zephyr that referenced this pull request Aug 28, 2025
@ceolin
west: mbedtls: Bump to 3.6.4
d03e3a7 
Bump mbetls to 3.6.4.

Fixes: zephyrproject-rtos#94596

Signed-off-by: Flavio Ceolin <[email protected]>
@ceolin ceolin dismissed stale reviews from valeriosetti, tomi-font, and kartben via d03e3a7 August 28, 2025 12:04
@zephyrbot zephyrbot added the Release Notes To be mentioned in the release notes label Aug 28, 2025
@zephyrbot zephyrbot added the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Aug 28, 2025
tomi-font
tomi-font previously requested changes Aug 28, 2025
View reviewed changes
Copy link
Contributor

@tomi-font tomi-font left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#94597 (review)

@tomi-font tomi-font removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Aug 28, 2025
@ceolin
west: mbedtls: Bump to 3.6.4
f9f2e67 
Bump mbetls to 3.6.4.

Fixes: zephyrproject-rtos#94596

Signed-off-by: Flavio Ceolin <[email protected]>
@ceolin ceolin requested a review from tomi-font August 28, 2025 14:00
@zephyrbot zephyrbot added the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Aug 28, 2025
@tomi-font
Copy link
Contributor

tomi-font commented Aug 28, 2025

I don't know why the bot keeps adding the DNM label. 🤔

@tomi-font tomi-font closed this Aug 28, 2025
@tomi-font tomi-font reopened this Aug 28, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 28, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@ceolin ceolin removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Sep 3, 2025
@carlescufi
Copy link
Member

carlescufi commented Sep 5, 2025

I don't know why the bot keeps adding the DNM label. 🤔

Because you have metadata changes in the module, i.e. you changed module.yml. In this case the DNM needs to be removed manually once the PR is ready, because it requires human inspection.

@nashif nashif merged commit eae415d into zephyrproject-rtos:v4.1-branch Sep 6, 2025
50 of 52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fabiobaltieri fabiobaltieri fabiobaltieri approved these changes

@valeriosetti valeriosetti valeriosetti approved these changes

@kartben kartben kartben left review comments

@tomi-font tomi-font Awaiting requested review from tomi-font

Assignees

@kartben kartben

@fabiobaltieri fabiobaltieri

Labels

manifest manifest-mbedtls Release Notes To be mentioned in the release notes size: XS A PR changing only a single line of code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

mbedtls: update to 3.6.4 in 4.1 branch

8 participants

@ceolin @zephyrbot @fabiobaltieri @tomi-font @carlescufi @kartben @valeriosetti @nashif